Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[UX] Improve empty library instructions and denied anticheat copy #4312

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[UX] Improve empty library instructions and denied anticheat copy #4312

wants to merge 1 commit into from
+39 −12

Conversation

arielj
Copy link
Collaborator

@arielj arielj commented Jan 28, 2025 •
edited
Loading

This PR improves two messages that were not super clear based on the number of questions in discord.

The message displayed in the library when Heroic is started the first time now explains a bit what the Log in and Add Game options do, so users can understand what to do to see their games.

image

The other message I changed is the one that we display when a user tries to install a game with broken/denied anticheat. Currently, the terms used in the copy are based on what AreWeAnticheatYet uses, but it's not clear for new users. Instead, now it explains what that actually means and also tells the user that there's nothing to do about the multiplayer/online features.

image

Use the following Checklist if you have changed something on the Backend or Frontend:

  • Tested the feature and it's working on a current and clean install.
  • Tested the main App features and they are still working on a current and clean install. (Login, Install, Play, Uninstall, Move games, etc.)
  • Created / Updated Tests (If necessary)
  • Created / Updated documentation (If necessary)

@arielj arielj added the pr:ready-for-review Feature-complete, ready for the grind! :P label Jan 28, 2025
@arielj arielj requested review from Etaash-mathamsetty, a team, flavioislima, CommandMC, Nocccer and imLinguin and removed request for a team January 28, 2025 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Etaash-mathamsetty Etaash-mathamsetty Awaiting requested review from Etaash-mathamsetty

@flavioislima flavioislima Awaiting requested review from flavioislima flavioislima was automatically assigned from Heroic-Games-Launcher/development

@CommandMC CommandMC Awaiting requested review from CommandMC CommandMC was automatically assigned from Heroic-Games-Launcher/development

@Nocccer Nocccer Awaiting requested review from Nocccer Nocccer was automatically assigned from Heroic-Games-Launcher/development

@imLinguin imLinguin Awaiting requested review from imLinguin imLinguin was automatically assigned from Heroic-Games-Launcher/development

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
pr:ready-for-review Feature-complete, ready for the grind! :P
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@arielj